Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix verifyPlugin Gradle task #919

Merged
merged 2 commits into from
Jun 26, 2024

Conversation

citizenmatt
Copy link
Member

This PR fixes the verifyPlugin Gradle task after migrating to v2 of the Gradle IntelliJ Plugin. It was broken due to disabling the useBinaryReleases build feature, which controls whether the referenced IDE is downloaded as a consumer binary release from the main CDN, or from Maven. The default is to use consumer binary releases, but the Plugin DevKit plugin wasn't able to download sources for these references during debugging, etc. The Plugin DevKit has now been updated, and supports downloading sources for binary releases in 2024.1.4 and the latest 242 EAP.

This PR also enables Gradle caching. Without this, Gradle appears to re-run the "extractor transformer" on each run, which seems to extract the IDE ready to be referenced. This means running tests can have additional overhead, up to 1 minute. Enabling caching removes this delay.

Binary releases can be used again now that the latest builds of 241 (2024.1.4) and 242 (EAP) include updates to the Plugin DevKit plugin that can download sources for binary releases. (Binary releases mean the referenced IDE is extracted and run from the same binaries as consumer releases).

This fixes verifyPlugin, which didn't support binary releases being disabled.
Without this, running tests could take up to 1 minute, mostly spent configuring Gradle. It appears that the extractor transformer would extract the IDE on each run.
@AlexPl292 AlexPl292 merged commit 135518e into JetBrains:master Jun 26, 2024
4 checks passed
@citizenmatt citizenmatt deleted the chore/gradle-plugin branch June 26, 2024 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants